Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update structured cloning for recent changes to HTML #1108

Merged
merged 1 commit into from
Apr 13, 2017

Conversation

domenic
Copy link
Contributor

@domenic domenic commented Apr 4, 2017

Fixes #1089.

It'd be good for someone with domain knowledge that this makes sense as defined for the internal slots.

@alvestrand
Copy link
Contributor

@henbos @martinthomson can you take a look?

Copy link
Contributor

@henbos henbos left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM except the Serializable should be moved to RTCCertificate.

webrtc.html Outdated
@@ -1321,7 +1328,7 @@
objects.</p>
<div>
<pre class="idl">
[ Constructor (optional RTCConfiguration configuration)]
[ Serializable, Constructor (optional RTCConfiguration configuration)]
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's the RTCCertificate that is Serializable, not the RTCPeerConnection?

@domenic domenic force-pushed the update-structured-clone branch from 6ecfa8d to ad9ca76 Compare April 10, 2017 20:33
@domenic
Copy link
Contributor Author

domenic commented Apr 10, 2017

Thanks, fixed!

@stefhak
Copy link
Contributor

stefhak commented Apr 13, 2017

LGTM.

@aboba aboba merged commit 13c67b0 into w3c:master Apr 13, 2017
@stefhak
Copy link
Contributor

stefhak commented May 11, 2017

@domenic : this PR was merged and then reverted via #1171. The reason for reverting was purely that we got pushback for referencing whatwg html (something we did already before, but it was detected now), and thus we reverted and referenced w3c html. I think everyone likes the new way to do "Safe passing of structured data" better, and I've raised an Issue on the html (5.2) spec (w3c/html#898) to introduce it in the w3c version.

@domenic
Copy link
Contributor Author

domenic commented May 11, 2017

Welp, if there's a conflict between technical excellence and bureacracy, I can't help you there. I tried my best.

@stefhak
Copy link
Contributor

stefhak commented May 12, 2017

We're grateful for that, and hope we can sort out the bureaucracy soon!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants